Add support for nested chart deps and repo prefix#69
Closed
shortwavedave wants to merge 118 commits intovmware-labs:mainfrom
shortwavedave:nested-deps
Closed
Add support for nested chart deps and repo prefix#69shortwavedave wants to merge 118 commits intovmware-labs:mainfrom shortwavedave:nested-deps
shortwavedave wants to merge 118 commits intovmware-labs:mainfrom
shortwavedave:nested-deps
Conversation
Fix typos and do some rewording in README.md
…h Helm plugin install
Added info command
Added tests for info command
Modify Installation script for Helm plugin
The version command prints two new lines at the end. This PR removes one of them.
Remove extra newline from version command
* Show warning when there are no images to pull
The output of `dt wrap` and `dt images pull` shows the following message
when there are no images to pull:
```
✔ All images pulled successfully
```
Instead, we are going to show a warning when there are no images to pull:
```
⚠️ No images found in Images.lock
```
* Add missing scenario
* Improve error flow
* Don't try to pull images if there are no images to pull
Bumps [github.com/lestrrat-go/jwx/v2](https://github.com/lestrrat-go/jwx) from 2.0.16 to 2.0.19. - [Release notes](https://github.com/lestrrat-go/jwx/releases) - [Changelog](https://github.com/lestrrat-go/jwx/blob/develop/v2/Changes) - [Commits](lestrrat-go/jwx@v2.0.16...v2.0.19) --- updated-dependencies: - dependency-name: github.com/lestrrat-go/jwx/v2 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
…com/lestrrat-go/jwx/v2-2.0.19 Bump github.com/lestrrat-go/jwx/v2 from 2.0.16 to 2.0.19
* Refactor cmd structure to easier external reuse
Add `dt auth login` and `dt auth logout` to login and logout from OCI registries
* Support authentication * Fixed function call * Add wrap auth and tests * Add unwrap auth and tests * Replace deprecated settings * Use temp credentials file * Remove duplicated log * Update dependencies * Remove username and password flags and refactor tests to use API for auth * Add container registry credentials to wrap * Test wrap with container registry credentials * Add container registry credentials to unwrap * Tidy dependencies --------- Co-authored-by: juanjo <jmedinagodoy@vmware.com>
* Refactor loggers * Move code to proper file
Bumps [helm.sh/helm/v3](https://github.com/helm/helm) from 3.14.0 to 3.14.1. - [Release notes](https://github.com/helm/helm/releases) - [Commits](helm/helm@v3.14.0...v3.14.1) --- updated-dependencies: - dependency-name: helm.sh/helm/v3 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [helm.sh/helm/v3](https://github.com/helm/helm) from 3.14.1 to 3.14.2. - [Release notes](https://github.com/helm/helm/releases) - [Commits](helm/helm@v3.14.1...v3.14.2) --- updated-dependencies: - dependency-name: helm.sh/helm/v3 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* feat: support multiple values yaml files * chore: remove unused parameters
Co-authored-by: Alejandro Moreno <amorenoc@vmware.com>
Bumps [github.com/go-jose/go-jose/v3](https://github.com/go-jose/go-jose) from 3.0.1 to 3.0.3. - [Release notes](https://github.com/go-jose/go-jose/releases) - [Changelog](https://github.com/go-jose/go-jose/blob/v3.0.3/CHANGELOG.md) - [Commits](go-jose/go-jose@v3.0.1...v3.0.3) --- updated-dependencies: - dependency-name: github.com/go-jose/go-jose/v3 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps gopkg.in/go-jose/go-jose.v2 from 2.6.2 to 2.6.3. --- updated-dependencies: - dependency-name: gopkg.in/go-jose/go-jose.v2 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps google.golang.org/protobuf from 1.32.0 to 1.33.0. --- updated-dependencies: - dependency-name: google.golang.org/protobuf dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [github.com/docker/docker](https://github.com/docker/docker) from 25.0.2+incompatible to 25.0.5+incompatible. - [Release notes](https://github.com/docker/docker/releases) - [Commits](moby/moby@v25.0.2...v25.0.5) --- updated-dependencies: - dependency-name: github.com/docker/docker dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Alejandro Moreno <amorenoc@vmware.com>
Co-authored-by: alemorcuq <alemorcuq@users.noreply.github.com>
* Revert "Prepare release v0.4.1 (#63)" This reverts commit 26fa9c2. * use go 1.21.9 for CVE-2023-45288 Signed-off-by: Alejandro Moreno <amorenoc@vmware.com> --------- Signed-off-by: Alejandro Moreno <amorenoc@vmware.com>
Co-authored-by: alemorcuq <alemorcuq@users.noreply.github.com>
Bumps [github.com/sigstore/cosign/v2](https://github.com/sigstore/cosign) from 2.2.3 to 2.2.4. - [Release notes](https://github.com/sigstore/cosign/releases) - [Changelog](https://github.com/sigstore/cosign/blob/main/CHANGELOG.md) - [Commits](sigstore/cosign@v2.2.3...v2.2.4) --- updated-dependencies: - dependency-name: github.com/sigstore/cosign/v2 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
|
@shortwavedave, you must sign our contributor license agreement before your changes are merged. Click here to sign the agreement. If you are a VMware employee, read this for further instruction. |
|
@shortwavedave, we have received your signed contributor license agreement. The review is usually completed within a week, but may take longer under certain circumstances. Another comment will be added to the pull request to notify you when the merge can proceed. |
|
@shortwavedave, VMware has approved your signed contributor license agreement. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This branch adds support for nested helm dependencies
This branch adds support for nested repository urls - the previous implementation only supported a single level
Resolves: #67